Skip to content

Conversation

@juanchoperezj
Copy link

What does this do?

Add new linting rules.

Why did you do this?

The objective is to reduce code review time by preventing changes that ESLint can address during development.
Card

Who/what does this impact?

Few existing LOCs were affected by this.

How did you test this?

Zero issues by running pnpm run lint.

@github-actions
Copy link

github-actions bot commented Dec 9, 2024

💯 Test Coverage

Lines Statements Branches Functions
Coverage: 54%
53.55% (271/506) 45.58% (98/215) 38.34% (74/193)

😎 Tests Results

Tests Skipped Failures Errors Time
70 0 💤 0 ❌ 0 🔥 27.667s ⏱️
👀 Tests Details • (54%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files53.5545.5838.3454.14 
api/common81.635062.584.78 
   utils.tsx85.294554.5490.3214–20
app0000 
   onboarding.tsx0100007–41
app/(app)0000 
   settings.tsx000013–18
ui77.7261.5369.2377.41 
   button.tsx10087.5100100126–132

Copy link
Collaborator

@fernandatoledo fernandatoledo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@guillermomachado guillermomachado force-pushed the feature/eslint-new-rules branch from fdd6d3b to 9757074 Compare January 10, 2025 13:57
@guillermomachado guillermomachado merged commit b89999f into master Jan 10, 2025
9 checks passed
@guillermomachado guillermomachado deleted the feature/eslint-new-rules branch January 10, 2025 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants